-
Notifications
You must be signed in to change notification settings - Fork 649
sentry: Enable tracing
feature
#3919
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
b1b6843
to
db6a254
Compare
☔ The latest upstream changes (presumably #3920) made this pull request unmergeable. Please resolve the merge conflicts. |
db6a254
to
0130a5a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change itself looks good but I'm not involved in the sentry part. If there are some agreements with Justin and/or Pietro about the log level, feel free to r=me.
18a73bc
to
344c753
Compare
☔ The latest upstream changes (presumably #3965) made this pull request unmergeable. Please resolve the merge conflicts. |
344c753
to
f85aac6
Compare
@pietroalbini @jtgeibel any objections on this? |
f85aac6
to
422c1a2
Compare
apparently not... 😅 @bors r=JohnTitor |
📌 Commit 422c1a2 has been approved by |
sentry: Enable `tracing` feature https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.2.21 enables us to use per-layer event filters, which previously blocked us from using the `tracing` feature of `sentry`. This PR enables the feature using the default configuration, which reports all `error!()` usages as errors and `warn!()` and `info!()` as breadcrumbs for associated errors.
💥 Test timed out |
422c1a2
to
b2bb38f
Compare
let's try this again after rebasing... @bors r=JohnTitor |
📌 Commit b2bb38f has been approved by |
☀️ Test successful - checks-actions |
https://github.com/tokio-rs/tracing/releases/tag/tracing-subscriber-0.2.21 enables us to use per-layer event filters, which previously blocked us from using the
tracing
feature ofsentry
. This PR enables the feature using the default configuration, which reports allerror!()
usages as errors andwarn!()
andinfo!()
as breadcrumbs for associated errors.